Skip to content

Conversation

@nael94
Copy link

@nael94 nael94 commented Jul 29, 2024

Hello there! I've discovered your repo while I'm looking for a powerful template engine for my new PHP MVC Framework. I really appreciate your effort to develop something like this!

During my testing, I've opened 13 issues to fix. As a thanks for you, I decided to say it on my way; I forked it and pushed my fixes and my special touches and ideas, you may review the seven closed issues on your repo to see what I'd done.

At the end, I tried to do my best to solve my all opened issues but I couldn't for now. However, I'll be happy to participate for any new ideas and fixes that you may need in the future.

Thanks for your effort to present this powerful framework.

[PS]: I've linted all files to reduce file sizes for a better stability and light loading.

nael94 added 8 commits July 29, 2024 11:49
…t set. For now, no need to write \ at the start and end of the namespace. Example: (App\Component) instead of (\App\Component\).
…mize extension for each component's view. See related issue comment.
…ner referring. See issue Corviz#17 comment for more info.
… dashed attributes won't be camelCased anymore due to ability to user custom data attributes, also introduced $this->getAttributesParsed() passed through custom component by $attributesParsed.
@nael94
Copy link
Author

nael94 commented Aug 10, 2024

I've also fixed issue #11 and changed the behavior of parsing attributes: dashed attributes won't be camelCased anymore due to ability to user custom data attributes, also introduced $this->getAttributesParsed() passed through custom component by $attributesParsed.

@nael94
Copy link
Author

nael94 commented Aug 18, 2024

For the commit id 7f140c8, this following case does not work anymore:

<x-alert :level="$message->type == 'error' ? 'danger' : 'info'">
    {{ $alertMessage }}
</x-alert>

The solution is to use @php @endphp directives to store the variable to use it here.
To solve this, it's in the $re regex parser in the code. I hope you help me solving this.

@nael94
Copy link
Author

nael94 commented Aug 18, 2024

Okay I've solved an issue: passing object or array as attribute parameter value in the custom component.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant